Skip to content
This repository has been archived by the owner on Sep 9, 2022. It is now read-only.

add iam_role_policy_attachment resource #489

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nakamasato
Copy link

@nakamasato nakamasato commented Nov 10, 2019

Why

What

  • Use iam_role_policy_attachment, iam_group_policy_attachment or iam_user_policy_attachment as the documentation suggests.
  • Only iam_role_policy_attachment is added in this PR

Check

  • Added resource: iam_role_policy_attachment

  • Class name: IAMRolePolicyAttachment

  • File name: iam_role_policy_attachment.rb

  • Command name: iamrpa

  • Executed terraform plan for my own resources and confirmed there's no changes

    terraform plan
    
    ...
    
    ------------------------------------------------------------------------
    
    No changes. Infrastructure is up-to-date.
    
    This means that Terraform did not detect any differences between your
    configuration and real physical resources that exist. As a result, no
    actions need to be performed.
    

Future work

Add iam_user_policy_attachment and iam_group_policy_attachment

@coveralls
Copy link

coveralls commented Nov 10, 2019

Coverage Status

Coverage remained the same at 100.0% when pulling 3269925 on nakamasato:iam_role_policy_attachment into 518879d on dtan4:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 4dc2401 on nakamasato:iam_role_policy_attachment into 518879d on dtan4:master.

@nakamasato nakamasato marked this pull request as ready for review November 10, 2019 13:48
@nakamasato nakamasato force-pushed the iam_role_policy_attachment branch from 4dc2401 to 3269925 Compare November 16, 2019 04:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants